fix: allow resolving public experts without PERSTACK_API_KEY#653
Merged
fix: allow resolving public experts without PERSTACK_API_KEY#653
Conversation
Public experts do not require authentication. Remove the early throw that blocked API calls without an API key, allowing the client to attempt the request (the API returns public experts without auth). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
resolveExpertToRunandresolveAllExpertsthat blocked API calls without an API keyAuthorizationheader when present)Changed files
packages/runtime/src/helpers/resolve-expert.ts— remove API key guardpackages/installer/src/expert-resolver.ts— remove API key guardpackages/runtime/src/helpers/resolve-expert.test.ts— update test to verify public expert resolution without keyTest plan
bun run typecheckpassesbun run testpassesresolve-expert.test.tsverifies public expert fetch without API keydocker run --rm -it -e ANTHROPIC_API_KEY perstack/perstack start create-expert "test"works without PERSTACK_API_KEY🤖 Generated with Claude Code